Conversation
PR Compliance Guide 🔍Below is a summary of compliance checks for this PR:
Compliance status legend🟢 - Fully Compliant🟡 - Partial Compliant 🔴 - Not Compliant ⚪ - Requires Further Human Verification 🏷️ - Compliance label |
||||||||||||||||||||||||
PR Code Suggestions ✨Explore these optional code suggestions:
|
|||||||||
Co-authored-by: adenchen123 <cykb0903@gmail.com> Co-authored-by: Aden <128471249+adenchen123@users.noreply.github.com> Co-authored-by: Jicheng Lu <franklujc@gmail.com> Co-authored-by: iceljc <46039045+iceljc@users.noreply.github.com> Co-authored-by: Jack <jjy_8888@163.com> Co-authored-by: jackjiang-sms <jianyong.jiang@starlight-sms.com>
|
reviewed |
PR Type
Enhancement
Description
Optimize null-coalescing assignment operator usage
Pass ResponseFormat parameter to Execute method
Improve code consistency and API completeness
Diagram Walkthrough
File Walkthrough
InstructService.Execute.cs
Simplify null-coalescing assignment operatorsrc/Infrastructure/BotSharp.Core/Instructs/Services/InstructService.Execute.cs
assignment operator (
??=)InstructModeController.cs
Add ResponseFormat parameter to Execute callsrc/Infrastructure/BotSharp.OpenAPI/Controllers/Instruct/InstructModeController.cs
responseFormatparameter to theExecutemethod callinput.ResponseFormatfrom the request to the instructor service